fix: include _noop tool in activeTools for LiteLLM proxy compatibility#9912
Conversation
The _noop dummy tool was being added to satisfy LiteLLM's requirement for a tools parameter when message history contains tool calls, but it was filtered out from activeTools. This caused compaction to fail with 'Bedrock doesn't support tool calling without tools= param' error.
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
|
The following comment was made by an LLM, it may be inaccurate: Potential Related PR Found:
Why it's related: This PR also addresses LiteLLM proxy compatibility issues with tool calling. While PR #9912 focuses specifically on the |
|
Thanks! |
|
I still have this problem. litellm.UnsupportedParamsError: Bedrock doesn't support tool calling without |
|
It works for me now in the latest version with: |
Thank you for your reply, this solved my problem.!!!!! |
Summary
Problem
When using
/compacton sessions with LiteLLM proxy providers, the request fails with:Root Cause
The
_noopdummy tool was being added correctly to satisfy LiteLLM's requirement, but it was filtered out fromactiveToolson line 211 ofllm.ts, so it was never actually sent in the API request.Fix
Remove
_noopfrom theactiveToolsfilter so it gets included in requests when needed.